Skip to content

refactor(controller): extract EnforceStrictPagination from JsonApiQueryAsync#128

Merged
Erlend Ellefsen (erlendellefsen) merged 1 commit into
mainfrom
refactor/extract-query-async-helpers
May 6, 2026
Merged

refactor(controller): extract EnforceStrictPagination from JsonApiQueryAsync#128
Erlend Ellefsen (erlendellefsen) merged 1 commit into
mainfrom
refactor/extract-query-async-helpers

Conversation

@erlendellefsen

@erlendellefsen Erlend Ellefsen (erlendellefsen) commented May 6, 2026

Copy link
Copy Markdown
Collaborator
  • Pulls the strict-pagination 404 check (19 lines) out of JsonApiQueryAsync into a private static helper.
  • No behavior change. Existing StrictPaginationIntegrationTests cover the path.
  • JsonApiQueryAsync shrinks from 86 to 67 lines and reads as a linear pipeline.

@erlendellefsen Erlend Ellefsen (erlendellefsen) merged commit e501631 into main May 6, 2026
2 checks passed
@erlendellefsen Erlend Ellefsen (erlendellefsen) deleted the refactor/extract-query-async-helpers branch May 6, 2026 13:25
Erlend Ellefsen (erlendellefsen) pushed a commit that referenced this pull request May 7, 2026
🤖 I have created a release *beep* *boop*
---


##
[2.1.0](v2.0.1...v2.1.0)
(2026-05-07)


### Features

* **docs:** migrate to mkdocs material with generated API reference
([#129](#129))
([765d46a](765d46a))
* **pagination:** return 404 when page exceeds total pages in strict
mode
([#126](#126))
([b639fdd](b639fdd))


### Bug Fixes

* correct casing for Intility.DotnetApiDocs in dotnet-tools
configuration
([01ed721](01ed721))
* follow dotnet api docs rename
([abe8436](abe8436))


### Code Refactoring

* **controller:** extract EnforceStrictPagination from JsonApiQueryAsync
([#128](#128))
([e501631](e501631))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: intility-release-bot[bot] <175299729+intility-release-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant